[feat] GameSetting 변경 기능 추가 #107

LimKangHyun · 2025-07-22T08:09:23Z

🛰️ Issue Number

Closes #100

🪐 작업 내용

게임 설정 변경 기능 구현

퀴즈 변경, 제한 시간 변경, 라운드 변경 메시지 매핑 추가
각 변경 요청을 GameService의 changeGameSetting 메서드로 처리
각 요청은 GAME_SETTING 을 브로드캐스팅하고,
퀴즈 변경에 대해서만, 방장을 제외한 모든 플레이어의 준비가 false가 되도록 하고, PLAYER_LIST를 추가로 브로드캐스팅합니다.
요청으로 들어온 변경사항이 기존의 값과 동일하다면 무시되어 브로드캐스팅 되지 않도록 구현했습니다.

// 변경 요청 퀴즈 아이디가 기존 퀴즈아이디와 같은 경우 처리 예시
if (Objects.equals(room.getGameSetting().getQuizId(), quizId)) {
    return false; // 동일하면 무시
}

toPlayerListResponse가 false로만 내려지는 부분을 Player의 레디상태로 가져오도록 수정했습니다.

GameSettingChanger 인터페이스 도입

if-else 분기문을 사용해 각 요청을 처리하게 되면, GameSetting 항목이 늘어날수록 코드가 복잡해지고 유지보수가 어려워지게 되어, GameSettingChanger 인터페이스를 도입하였고, 각 DTO가 이 인터페이스를 구현하여 자신만의 변경 로직을 갖도록 설계하였습니다.
클래스는 늘어나지만, 만약 GameSetting 항목이 늘어나게 되면, changeGameSetting 메서드는 건드리지 않고, 새로운 DTO만 확장하면 되어서 OCP가 지켜지는 장점이 있을 것 같습니다.
afterChange 메서드는 각 요청에 대해서 고유한 동작을 수행합니다.
(퀴즈 요청은 다른 요청과 다르게, 플레이어의 준비상태를 전부 false로 만들고, PLAYER_LIST를 브로드캐스팅해야합니다.)

메서드 이동

플레이어 레디와 관련된 부분을 RoomService에서 GameService로 이동시켰습니다.

빈 등록

RoomUpdatedEventListener, RoomDeletedEventListener 빈 등록 했습니다.

📚 Reference

✅ Check List

코드가 정상적으로 컴파일되나요?
테스트 코드를 통과했나요?
merge할 브랜치의 위치를 확인했나요?
Label을 지정했나요?

- GameSettingChanger 인터페이스 도입

sehee123 · 2025-07-22T16:09:34Z

backend/src/main/java/io/f1/backend/domain/game/app/GameService.java

+    public void handlePlayerReady(Long roomId, String sessionId) {
+        Player player =
+                roomRepository
+                        .findPlayerInRoomBySessionId(roomId, sessionId)


이렇게 player를 가져오는 코드가 있는줄은 몰랐네요!

단순하게 room.PlayerSessionMap.get(sessionId) 를 쓰지 않고 이렇게 가져오는 이유가 있을까요?
NPE 때문일까요?

저건 작성한지 좀 된것같은데, 세션아이디 재연결 로직이 없어서 저렇게 짠것 같습니다. 재연결 로직에서 맵에서 바로 세션을 바로 삭제하지 않으니까 Room안에서 플레이어는 null일 가능성이 없을것 같네요!
소켓의 세션아이디가 널이 아니어도 룸에 플레이어로 등록되지 않은 경우도 존재할 것 같네요! null 체크하나 추가해서 수정했습니다!
수정했습니다!

sehee123 · 2025-07-22T16:14:09Z

backend/src/main/java/io/f1/backend/domain/game/app/GameService.java

+        if (room.isPlaying()) {
+            throw new CustomException(RoomErrorCode.GAME_ALREADY_PLAYING);
+        }
+        if (!Objects.equals(player.getId(), room.getHost().getId())) {


오홍 이것도 NPE 를 방지하고자 Objects.equals를 쓰신건가요?

Host에 대해서는 널 체크를 하지 않아도 될것 같습니다! isHost로 변경했습니다!

sehee123 · 2025-07-22T16:17:56Z

backend/src/main/java/io/f1/backend/domain/game/dto/request/QuizChangeRequest.java

+        int questionSize = quiz.getQuestions().size();
+        room.getGameSetting().changeQuiz(quiz);
+        // 퀴즈의 문제 갯수로 변경
+        room.getGameSetting().changeRound(questionSize, questionSize);


room.getGameSetting().changeQuiz()
room.getGameSetting().changeRound() 를
Room 객체 안에 하나의 메소드로 만들면 깔끔할 것 같다는 의견입니다!

gameSetting없이 Room으로 호출가능하도록 변경했습니다!

sehee123 · 2025-07-22T16:19:28Z

backend/src/main/java/io/f1/backend/domain/game/dto/request/QuizChangeRequest.java

+    public void afterChange(Room room, MessageSender messageSender) {
+        room.resetAllPlayerReadyStates();
+
+        String destination = "/sub/room/" + room.getId();


WebsocketUtils 에서 getDestination(roomId)로 대체할 수 있을 것 같습니다!

sehee123 · 2025-07-22T16:24:51Z

backend/src/main/java/io/f1/backend/domain/game/dto/request/TimeLimit.java

+        return Arrays.stream(values())
+                .filter(t -> t.value == value)
+                .findFirst()
+                .orElseThrow(() -> new CustomException(GameErrorCode.GAME_SETTING_CONFLICT));


에러 내용을 조금 세분화해서 유효하지 않은 타이머 설정입니다.
같은 타이머 에러라는걸 명시하면 어떨까요?.?

게임 내에서 일반 사용자가 유효하지 않은 타이머를 보낼일은 없다고 생각해서, 비정상적인 요청에 대해서 만들어놓은 방어적 예외라 예외 내용을 구체화하지는 않았습니다!

sehee123

수고하셨습니다! 👍

jiwon1217 · 2025-07-23T03:04:18Z

backend/src/main/java/io/f1/backend/domain/game/dto/request/TimeLimitChangeRequest.java

+
+    @Override
+    public boolean change(Room room, QuizService quizService) {
+        if (room.getGameSetting().getTimeLimit() == timeLimit) {


Room 내부 메서드로 위치시키면 좋을 것 같습니다 !

참고링크: https://mangkyu.tistory.com/147

room과 gameSetting의 관계를 생각을 안하고 만들었네요 수정했습니다!
인사이트 감사합니다!

jiwon1217 · 2025-07-23T03:04:36Z

backend/src/main/java/io/f1/backend/domain/game/dto/request/TimeLimitChangeRequest.java

+        if (room.getGameSetting().getTimeLimit() == timeLimit) {
+            return false; // 동일하면 무시
+        }
+        room.getGameSetting().changeTimeLimit(TimeLimit.from(timeLimit));


위의 리뷰 내용과 동일합니다 !

jiwon1217 · 2025-07-23T03:13:15Z

backend/src/main/java/io/f1/backend/domain/game/model/Room.java

+
+    public void resetAllPlayerReadyStates() {
+        for (Player player : playerSessionMap.values()) {
+            if (Objects.equals(player.getId(), getHost().getId())) continue;


isHost() 메서드가 아닌 Objects.equal()를 통해 직접 비교한 이유가 궁금합니다 ! 만약, 이유가 null-safe라면 isHost()를 null-safe하게 바꿔 사용하는 것이 가독성이나 일관성 측면에서 좋을 것 같습니다 !

Host에 대해서는 널 체크를 하지 않아도 될것 같습니다! isHost로 변경했습니다!

backend/src/main/java/io/f1/backend/domain/game/dto/request/QuizChangeRequest.java

backend/src/main/java/io/f1/backend/domain/game/app/GameService.java

jiwon1217 · 2025-07-23T10:43:21Z

backend/src/main/java/io/f1/backend/domain/game/app/GameService.java

+        Room room = findRoom(roomId);

-        return playerSessionMap.values().stream().allMatch(Player::isReady);
+        Player player = room.getPlayerSessionMap().get(sessionId);


이 부분도 Room의 메서드로 분리할 수 있을 것 같습니다 !

Room클래스 안에, getPlayerBySessionId 만들어서 교체했습니다!

jiwon1217

고생하셨습니다 ! OCP를 고려하여 설계한 부분이 인상깊네요

silver-eunjoo

확인했습니다 !! 인터페이스로 구현하신 방식이 굉장하십니다 ! 배워갑니다..! 고생하셨습니다 :)

* ✨ feat: 게임 설정 변경 인터페이스 및 구현체 추가 - GameSettingChanger 인터페이스 도입 * chore: Java 스타일 수정 * ✨ feat: 게임 설정 변경 기능 추가 * chore: Java 스타일 수정 * 🚚 move: 소켓 요청 DTO request 디렉토리로 이동 * 🚚 move: 소켓 요청 DTO 경로 변경 * chore: Java 스타일 수정 * ♻️ refactor: 게임 세팅 요청 후처리 분리 * chore: Java 스타일 수정 * ♻️ refactor: 코드 리뷰 반영 * chore: Java 스타일 수정 * 🗑️ remove: 불필요 코드 삭제 * chore: Java 스타일 수정 * ♻️ refactor: isHost 원상복구 * ♻️ refactor: QuizChangeRequest, RoundChangeRequest 타입 수정 * 🔧 chore: RoomUpdatedEventListener, RoomDeletedEventListener 빈 등록 * chore: Java 스타일 수정 * chore: Java 스타일 수정 * ♻️ refactor: player 조회 메서드 추가 --------- Co-authored-by: github-actions <>

* ✨ feat : 게임 종료 로직 구현 * chore: Java 스타일 수정 * ✨ feat: 게임 종료 후 disconnected 참여자 정리 * 🔧 chore : 깃 충돌 해결 * chore: Java 스타일 수정 * ♻️ refactor : 게임 시작 시, 실시간 랭킹도 함께 SEND * ♻️ refactor : 서비스 간의 순환참조 해결 * chore: Java 스타일 수정 * ♻️ refactor : 게임 종료 시 로직 변경 * chore: Java 스타일 수정 * 🔧 chore: 게임 종료 시 모든 메세지 브로드캐스팅하도록 변경 * chore: Java 스타일 수정 * 🐛 fix: index 예외, Lazy 예외 버그 수정 * chore: Java 스타일 수정 * ✨ [feat] 유저 닉네임으로 정보 조회 (#101) * ✨ feat: 특정 닉네임이 포함된 유저 정보 조회 기능 * 🐛 fix: 닉네임 중복 확인 시 대소문자 구분하지 않음 * chore: Java 스타일 수정 --------- Co-authored-by: github-actions <> * ♻️ refactor: 세션 타임아웃 값을 환경변수로 대치 (#105) * 🔧 chore : 깃 충돌 해결 * 🔧 chore : 깃 충돌 해결 * ♻️ refactor: 랭킹 조회 Redis 적용 (#111) * ✨ feat: GameSetting 변경 기능 추가 (#107) * ✨ feat: 게임 설정 변경 인터페이스 및 구현체 추가 - GameSettingChanger 인터페이스 도입 * chore: Java 스타일 수정 * ✨ feat: 게임 설정 변경 기능 추가 * chore: Java 스타일 수정 * 🚚 move: 소켓 요청 DTO request 디렉토리로 이동 * 🚚 move: 소켓 요청 DTO 경로 변경 * chore: Java 스타일 수정 * ♻️ refactor: 게임 세팅 요청 후처리 분리 * chore: Java 스타일 수정 * ♻️ refactor: 코드 리뷰 반영 * chore: Java 스타일 수정 * 🗑️ remove: 불필요 코드 삭제 * chore: Java 스타일 수정 * ♻️ refactor: isHost 원상복구 * ♻️ refactor: QuizChangeRequest, RoundChangeRequest 타입 수정 * 🔧 chore: RoomUpdatedEventListener, RoomDeletedEventListener 빈 등록 * chore: Java 스타일 수정 * chore: Java 스타일 수정 * ♻️ refactor: player 조회 메서드 추가 --------- Co-authored-by: github-actions <> * 🐛 fix: 라운드 변경 시 question개수 조회에 fetch join 적용 (#116) * 🐛 fix: round 변경 시 question 조회에 fetch join 적용 * ♻️ refactor: questionCount만 찾는 쿼리로 변경 및 불필요한 퀴즈 아이디 조회 삭제 * chore: Java 스타일 수정 --------- Co-authored-by: github-actions <> * chore: Java 스타일 수정 * chore: Java 스타일 수정 * 🔧 chore : 테스트 코드 통과 * chore: Java 스타일 수정 --------- Co-authored-by: github-actions <> Co-authored-by: Jiwon Kwak <[email protected]> Co-authored-by: dlsrks1021 <[email protected]> Co-authored-by: kanghyun <[email protected]>

LimKangHyun and others added 7 commits July 22, 2025 16:49

✨ feat: 게임 설정 변경 인터페이스 및 구현체 추가

be3f034

- GameSettingChanger 인터페이스 도입

chore: Java 스타일 수정

e1d79fc

✨ feat: 게임 설정 변경 기능 추가

214bc54

chore: Java 스타일 수정

ccac167

🚚 move: 소켓 요청 DTO request 디렉토리로 이동

9af131a

🚚 move: 소켓 요청 DTO 경로 변경

2fb0b8f

chore: Java 스타일 수정

a7ead79

LimKangHyun linked an issue Jul 22, 2025 that may be closed by this pull request

[feat] 게임 시작 전, GameSetting 변경 기능 #100

Closed

2 tasks

LimKangHyun changed the title ~~Feat/100~~ [feat] GameSetting 변경 기능 Jul 22, 2025

LimKangHyun changed the title ~~[feat] GameSetting 변경 기능~~ [feat] GameSetting 변경 기능 추가 Jul 22, 2025

LimKangHyun self-assigned this Jul 22, 2025

LimKangHyun requested review from dlsrks1021, jiwon1217, sehee123 and silver-eunjoo and removed request for silver-eunjoo July 22, 2025 08:10

LimKangHyun added the enhancement New feature or request label Jul 22, 2025

LimKangHyun and others added 2 commits July 23, 2025 00:55

♻️ refactor: 게임 세팅 요청 후처리 분리

6981459

chore: Java 스타일 수정

fdaaee7

sehee123 reviewed Jul 22, 2025

View reviewed changes

sehee123 approved these changes Jul 22, 2025

View reviewed changes

jiwon1217 reviewed Jul 23, 2025

View reviewed changes

♻️ refactor: 코드 리뷰 반영

655929e

github-actions and others added 4 commits July 23, 2025 04:22

chore: Java 스타일 수정

39d9f54

🗑️ remove: 불필요 코드 삭제

80101d0

chore: Java 스타일 수정

db62cbc

♻️ refactor: isHost 원상복구

181f4ab

dlsrks1021 reviewed Jul 23, 2025

View reviewed changes

backend/src/main/java/io/f1/backend/domain/game/dto/request/QuizChangeRequest.java Outdated Show resolved Hide resolved

backend/src/main/java/io/f1/backend/domain/game/app/GameService.java Show resolved Hide resolved

LimKangHyun and others added 6 commits July 23, 2025 17:55

♻️ refactor: QuizChangeRequest, RoundChangeRequest 타입 수정

c7506d9

🔧 chore: RoomUpdatedEventListener, RoomDeletedEventListener 빈 등록

b6ae413

chore: Java 스타일 수정

476ceea

🔧 fix: 충돌 해결

b7852c2

Merge remote-tracking branch 'origin/feat/100' into feat/100

2db2e95

chore: Java 스타일 수정

7c6515e

dlsrks1021 approved these changes Jul 23, 2025

View reviewed changes

jiwon1217 reviewed Jul 23, 2025

View reviewed changes

jiwon1217 approved these changes Jul 23, 2025

View reviewed changes

♻️ refactor: player 조회 메서드 추가

482bc72

LimKangHyun merged commit 2d23468 into dev Jul 24, 2025
2 checks passed

silver-eunjoo reviewed Jul 24, 2025

View reviewed changes

[feat] GameSetting 변경 기능 추가 #107

[feat] GameSetting 변경 기능 추가 #107

Uh oh!

Conversation

LimKangHyun commented Jul 22, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🛰️ Issue Number

🪐 작업 내용

게임 설정 변경 기능 구현

GameSettingChanger 인터페이스 도입

메서드 이동

빈 등록

📚 Reference

✅ Check List

Uh oh!

sehee123 Jul 22, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

LimKangHyun Jul 23, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

LimKangHyun Jul 23, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sehee123 Jul 22, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sehee123 Jul 22, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sehee123 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

LimKangHyun Jul 23, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

LimKangHyun Jul 24, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jiwon1217 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

silver-eunjoo left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

LimKangHyun commented Jul 22, 2025 •

edited

Loading

sehee123 Jul 22, 2025 •

edited

Loading

LimKangHyun Jul 23, 2025 •

edited

Loading

LimKangHyun Jul 23, 2025 •

edited

Loading

sehee123 Jul 22, 2025 •

edited

Loading

sehee123 Jul 22, 2025 •

edited

Loading

LimKangHyun Jul 23, 2025 •

edited

Loading

LimKangHyun Jul 24, 2025 •

edited

Loading